Skip to content

Conversation

@ldematte
Copy link
Contributor

Backports the following commits to 9.0:

…ke 2) (elastic#124429)

Writing tests for elastic#123861, turns out that elastic#124195 is not enough.
We really need new IT test cases for "always allowed" actions: in order to be sure they are allowed, we need to setup the plugin with no policy.
This PR adds test cases for that, plus the support for writing test functions that accept one Environment parameter: many test paths we test and allow/deny are relative to paths in Environment, so it's useful to have access to it (see readAccessConfigDirectory as an example)
@ldematte ldematte requested a review from a team as a code owner March 12, 2025 08:46
@ldematte ldematte added :Core/Infra/Entitlements Entitlements infrastructure >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Core/Infra Meta label for core/infra team labels Mar 12, 2025
@elasticsearchmachine elasticsearchmachine merged commit 183221d into elastic:9.0 Mar 12, 2025
16 checks passed
@ldematte ldematte deleted the backport/9.0/pr-124429 branch March 12, 2025 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v9.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants